Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SIG-scalability charter. #1607

Closed
wants to merge 1 commit into from
Closed

Conversation

porridge
Copy link
Member

SIG-scalability leads:
/cc @wojtek-t @countspongebob
Steering committee representatives:
/cc @thockin @smarterclayton @spiffxp

@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jan 15, 2018
@k8s-ci-robot k8s-ci-robot requested a review from jbeda January 15, 2018 15:27
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: porridge
We suggest the following additional approver: wojtek-t

Assign the PR to them by writing /assign @wojtek-t in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@k8s-github-robot k8s-github-robot added the sig/scalability Categorizes an issue or PR as relevant to SIG Scalability. label Jan 15, 2018
Copy link
Contributor

@countspongebob countspongebob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a good update, and LGTM. However, we should have some discussion about how to interact with people with small amounts of time to contribute.

@thockin
Copy link
Member

thockin commented Jan 19, 2018

lgtm

@wojtek-t
Copy link
Member

LGTM

@countspongebob - I agree that we may be able to tweak it in the future, but I think it's great starting point.

@porridge
Copy link
Member Author

ping @smarterclayton @spiffxp

@wojtek-t
Copy link
Member

@bgrant0607 @jdumars - FYI if you have any suggestions

@porridge
Copy link
Member Author

@smarterclayton @spiffxp please comment
This is my last week working on kubernetes, so I'd like to get this submitted.

@spiffxp
Copy link
Member

spiffxp commented Feb 13, 2018

Things that are missing from the charter:

  • what are the responsibilities of your SIG leads?
  • what process do you use to change SIG leadership?
  • what code do you own?

I'm not a fan of blocking the entire queue, but I guess I'm fighting a losing battle here.

@porridge
Copy link
Member Author

These are very good questions, none of which I know answers to.
@shyamjvs @wojtek-t @countspongebob can you comment on the above?
@wasylkowski-a FYI we'll probably need a new owner for this PR since it's unlikely we will be able to find an agreed upon answer to this question, get the final approvals and get it merged in the remaining 3 days.

@wasylkowski-a
Copy link

Re. the PR I will take it over if needed, but let's discuss the course of action offline.

@bgrant0607
Copy link
Member

FYI, we're working on a SIG charter template here:
#1650

It's still missing things like scope of the SIG.

Please also make sure your subprojects are correctly identified in https://github.com/kubernetes/community/blob/master/sigs.yaml

@wasylkowski-a
Copy link

I have created a copy (#1829) where we should continue given that this PR will not be updated anymore (see @porridge's comment).

@bgrant0607
Copy link
Member

cc @kubernetes/steering-committee

@porridge porridge closed this Mar 10, 2018
@porridge porridge deleted the charter branch March 10, 2018 12:41
@wojtek-t
Copy link
Member

I've openede #2149 as a replacement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. sig/scalability Categorizes an issue or PR as relevant to SIG Scalability. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants